-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Using remotes before remotes is installed #67
Conversation
Assumption this is why we have this error: https://github.com/epijim/episheet/actions/runs/7357960789/job/20030432815 Signed-off-by: James Black <[email protected]>
Signed-off-by: James Black <[email protected]>
Signed-off-by: James Black <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix!
Slowly finding more :) issues, will move to draft sorry @cicdguy so far:
|
Signed-off-by: James Black <[email protected]>
Signed-off-by: James Black <[email protected]>
Signed-off-by: James Black <[email protected]>
@@ -1,14 +1,17 @@ | |||
#!/usr/bin/env Rscript | |||
|
|||
# Install remotes | |||
install.packages("remotes") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out that pak
is more efficient and will eventually replace remotes
. What do you think about swapping out remotes
with pak
?
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA 1 out of 2 committers have signed the CLA. |
Assumption this is why we have this error: https://github.com/epijim/episheet/actions/runs/7357960789/job/20030432815